Skip to content

net: fix multi-client connection issues#351

Open
leonardocavagnis wants to merge 3 commits intoarduino:mainfrom
leonardocavagnis:portentaH7_conf_network
Open

net: fix multi-client connection issues#351
leonardocavagnis wants to merge 3 commits intoarduino:mainfrom
leonardocavagnis:portentaH7_conf_network

Conversation

@leonardocavagnis
Copy link
Copy Markdown
Member

This PR updates the default Zephyr configuration for the Arduino Portenta H7 to properly support multiple simultaneous TCP connections.

The previous configuration used conservative defaults that limited the network stack's ability to handle more than one active socket alongside the listening server.

This PR resolves the issues arising from #227, where only a single client could connect at a time when using the AdvancedChatServer.ino example, preventing Connection refused errors for subsequent clients.

@per1234 per1234 added the bug Something isn't working label Feb 22, 2026
@pennam pennam force-pushed the portentaH7_conf_network branch from e093c94 to 3af2311 Compare March 4, 2026 10:05
@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Mar 18, 2026

CLA assistant check
All committers have signed the CLA.

@pennam pennam force-pushed the portentaH7_conf_network branch from 3af2311 to 4a39e1d Compare April 30, 2026 09:16
@pennam pennam changed the title Portenta H7: fix multi-client connection issues net: fix multi-client connection issues Apr 30, 2026
@pennam
Copy link
Copy Markdown

pennam commented Apr 30, 2026

extended fix for OPTA and portenta C33

Tested together with #227 on portenta h7 portenta c33 OPTA and GIGA

RAM consumption increased of 5kB

@github-actions
Copy link
Copy Markdown

Built 0.55.1-0.dev+4a39e1dc

CI run PASSED 🟢

ArtifactBoardCoreTestsRAMSketchesWarningsErrors
zephyr_contrib ek_ra8d1 📗

11.9%

2--
frdm_mcxn947 6 🏷️

58.0%

2--
frdm_rw612 2 🏷️

83.0%

2--
✔️* zephyr_main giga 4 🏷️ ✅*

54.6%

5620-
nano33ble 1 🏷️ ✅*

78.8%

2810-
nano_matter 📗 ✔️*

⚠️ 85.7%

208(2*)
nicla_vision 4 🏷️ ✔️*

56.9%

5012(10*)
niclasense 2 🏷️ ✅*

⚠️ 87.4%

2610-
opta 4 🏷️ ✅*

57.7%

6034-
portentac33 3 🏷️ ✔️*

⚠️ 95.8%

6226(8*)
portentah7 3 🏷️ ✅*

58.4%

7034-
✅* zephyr_unoq unoq 📗 ✅*

26.9%

748-
Legend

BoardTestStatus description
🔥 🔥 Test run failed to complete.
🔴 Test completed with unexpected errors.
✔️* 🚫 Test completed with errors, but all are known/expected.
✅* 🟡 Test completed with some warnings; no errors detected.
🟢 Test passed successfully, with no warnings or errors.
🌑 🌑 Test was skipped.

@pennam pennam requested a review from andreagilardoni April 30, 2026 10:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants